Skip to content

fix(@angular/cli): correct dev dependency detection logic in ng add#32638

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:add-deps
Mar 2, 2026
Merged

fix(@angular/cli): correct dev dependency detection logic in ng add#32638
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:add-deps

Conversation

@alan-agius4
Copy link
Collaborator

Previously, the logic for determining whether to install a package as a dev dependency in ng add was using a negative check (!== 'dependencies'). This has been changed to an explicit check (=== 'devDependencies') to ensure the same behaviour as previous versions.

Closes #32630

Previously, the logic for determining whether to install a package as a dev dependency in `ng add` was using a negative check (`!== 'dependencies'`). This has been changed to an explicit check (`=== 'devDependencies'`) to ensure the same behaviour as previous versions.

Closes angular#32630
@alan-agius4 alan-agius4 requested a review from clydin March 2, 2026 08:29
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Mar 2, 2026
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Mar 2, 2026
@alan-agius4 alan-agius4 merged commit 038a931 into angular:main Mar 2, 2026
69 of 75 checks passed
@alan-agius4 alan-agius4 deleted the add-deps branch March 2, 2026 14:09
@alan-agius4
Copy link
Collaborator Author

This PR was merged into the repository. The changes were merged into the following branches:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/cli target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(schematics): adding package to devDependencies

2 participants